Skip to content

Conversation

kraj
Copy link
Contributor

@kraj kraj commented Jan 27, 2024

No description provided.

kraj and others added 3 commits January 27, 2024 15:00
…inition

GCC-14 will treat implicit function declarations as errors. Therefore
include the required header to fix

error: implicit declaration of function 'device_bind_driver_to_node' [-Wimplicit-function-declaration]

Signed-off-by: Khem Raj <[email protected]>
While compiling rtl8169.c, There are many "make pointer from
integer without a cast" compile warnings. fix them with
adding cast.

Signed-off-by: Minda Chen <[email protected]>
Reviewed-by: Ramon Fried <[email protected]>
Reviewed-by: Leo Yu-Chi Liang <[email protected]>
GCC-14 find more warnings like
"make pointer from integer without a cast"
fix them by adding a type cast.

Signed-off-by: Khem Raj <[email protected]>
@plettich
Copy link

Can confirm that this patch works with current riscv64-linux-gnu-gcc (GCC) 14.1.0 on an Arch based system. The VisionFive2 boots just fine with the updated u-boot and OpenSBI JH7110_VF2_6.6_v5.12.0 from eMMC.

@wxjstz
Copy link

wxjstz commented Oct 18, 2025

I encountered the same situation when compiling.

There are a lot of warnings when compiling

include/configs/starfive-visionfive2.h:332:62: warning: backslash and newline separated by space
  332 |                         "setenv bootenv uEnv_Lite_emmc.txt;" \

The following patches are required

diff --git a/include/configs/starfive-visionfive2.h b/include/configs/starfive-visionfive2.h
index 0f681941b5..ec225711b8 100644
--- a/include/configs/starfive-visionfive2.h
+++ b/include/configs/starfive-visionfive2.h
@@ -329,7 +329,7 @@
                    "if test ${emmc_size} = 0; then "   \
                        "setenv bootenv uEnv_Lite.txt;"    \
                    "else "                                \
-                       "setenv bootenv uEnv_Lite_emmc.txt;" \  
+                       "setenv bootenv uEnv_Lite_emmc.txt;" \
                     "fi; "                                 \
                "fi;" \
                "run load_distro_uenv; " \

Copy link

@wxjstz wxjstz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This patch also need to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants